Skip to content

WI #2856 Add the rows count of all expanded OCCURS in the GetDataLayoutRequest result (CSV mode only)#2857

Merged
efr15 merged 4 commits into
developfrom
2856-add-the-rows-count-of-all-expanded-occurs-in-the-getdatalayoutrequest-result-csv-mode-only
Jun 4, 2026
Merged

WI #2856 Add the rows count of all expanded OCCURS in the GetDataLayoutRequest result (CSV mode only)#2857
efr15 merged 4 commits into
developfrom
2856-add-the-rows-count-of-all-expanded-occurs-in-the-getdatalayoutrequest-result-csv-mode-only

Conversation

@efr15

@efr15 efr15 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Fixes #2856

Regarding the LSR test change:

  • In the 1st call the original source is almost identical to simplePgm.cbl in ProcessorTests/DataLayout. The only difference is the missing data declaration in the LINKAGE SECTION. The expanded rows counts differ consequently by 1.
  • In the 2nd call the source is modified by removing 2 data declarations. The expanded rows count is consequently decreased by 2.

@efr15 efr15 self-assigned this Jun 1, 2026
@efr15 efr15 linked an issue Jun 1, 2026 that may be closed by this pull request
Add the rows count of all expanded OCCURS in the GetDataLayoutRequest result (CSV mode only) #2856
Closed
@efr15
efr15 requested a review from fm-117 June 1, 2026 13:36
fm-117
fm-117 requested changes Jun 2, 2026
View reviewed changes
Comment thread TypeCobol.LanguageServer/Processor/DataLayoutProcessor.cs
@efr15
efr15 requested a review from fm-117 June 2, 2026 15:13
@efr15
efr15 merged commit 4dba6c6 into develop Jun 4, 2026
8 checks passed
@efr15 efr15 mentioned this pull request Jul 23, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fm-117 fm-117 fm-117 approved these changes

Assignees

@efr15 efr15

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add the rows count of all expanded OCCURS in the GetDataLayoutRequest result (CSV mode only)

2 participants

@efr15 @fm-117